Skip to content

fix: Aegis 홈페이지 url 수정 및 매직넘버 상수로 처리 - #174

Merged
KwonDeaGeun merged 2 commits into
developfrom
fix/modify-aegis-homepage-url
Feb 14, 2026
Merged

fix: Aegis 홈페이지 url 수정 및 매직넘버 상수로 처리#174
KwonDeaGeun merged 2 commits into
developfrom
fix/modify-aegis-homepage-url

Conversation

@KwonDeaGeun

@KwonDeaGeun KwonDeaGeun commented Feb 14, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

릴리스 노트

  • Refactor

    • 로그인 페이지의 내부 구조를 개선했습니다.
  • Style

    • CSS 코드 스타일 표준화를 적용했습니다.

@KwonDeaGeun KwonDeaGeun self-assigned this Feb 14, 2026
@coderabbitai

coderabbitai Bot commented Feb 14, 2026

Copy link
Copy Markdown

개요

이 변경사항은 로그인 페이지의 하드코딩된 URL을 상수로 추출하고, 다이얼로그 컴포넌트의 클래스명 전파를 유지하며, CSS 파일의 따옴표 스타일을 표준화합니다.

변경사항

집단 / 파일(들) 요약
스타일 및 형식 최적화
src/components/ui/dialog.tsx, src/index.css
다이얼로그 컴포넌트의 className 전파 유지 및 CSS 플러그인 선언에서 따옴표 스타일 일관성화.
상수 추출 리팩토링
src/pages/LoginPage.tsx
하드코딩된 URL(GOOGLE_LOGIN_URL, AEGIS_HOMEPAGE_URL, GMAIL_GUIDE_URL)을 최상위 상수로 추출하여 유지보수성 개선.
🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목이 주요 변경사항을 명확하게 설명합니다. LoginPage에서 하드코딩된 URL들을 상수로 추출하고 Aegis 홈페이지 URL을 수정한 내용이 제목에 정확히 반영되어 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into develop

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


No actionable comments were generated in the recent review. 🎉


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@KwonDeaGeun
KwonDeaGeun merged commit b816543 into develop Feb 14, 2026
2 checks passed
@KwonDeaGeun
KwonDeaGeun deleted the fix/modify-aegis-homepage-url branch February 14, 2026 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant